Skip to content

[refactor-mutation-280] ๐Ÿ”จ Refactor : mutation ์ฝ”๋“œ ๊ฐœ์„  ๋ฐ eslint ์„ค์ • ์ถ”๊ฐ€#281

Closed
goodaseul wants to merge 2 commits into
devfrom
refactor/mutation-280
Closed

[refactor-mutation-280] ๐Ÿ”จ Refactor : mutation ์ฝ”๋“œ ๊ฐœ์„  ๋ฐ eslint ์„ค์ • ์ถ”๊ฐ€#281
goodaseul wants to merge 2 commits into
devfrom
refactor/mutation-280

Conversation

@goodaseul

Copy link
Copy Markdown
Collaborator

์š”์•ฝ

  • ๋ณ€๊ฒฝ ๋ชฉ์ (์™œ?): ๋ชฉํ‘œ ์‚ญ์ œ mutation์˜ ๋ถˆํ•„์š”ํ•œ ์ฝ”๋“œ ์ œ๊ฑฐ ๋ฐ ํ• ์ผ ํ† ๊ธ€ UX ๊ฐœ์„ , eslint ๋นŒ๋“œ ์—๋Ÿฌ ํ•ด๊ฒฐ
  • ์ฃผ์š” ๋ณ€๊ฒฝ(๋ฌด์—‡์„?): ๋‚™๊ด€์  ์—…๋ฐ์ดํŠธ ์ œ๊ฑฐ / ๋‚™๊ด€์  ์—…๋ฐ์ดํŠธ ์ ์šฉ / onSettled ํ†ตํ•ฉ / eslint ๊ทœ์น™ ์ถ”๊ฐ€

๋ณ€๊ฒฝ ๋‚ด์šฉ

  • UI/์ปดํฌ๋„ŒํŠธ
  • ๋กœ์ง/์œ ํ‹ธ
  • ๋ฌธ์„œ/์„ค์ •

eslint ์„ค์ • ์ถ”๊ฐ€ ์ƒ์„ธ

5๊ฐœ์›” ๋งŒ์— ํ”„๋กœ์ ํŠธ๋ฅผ ์žฌ์‹คํ–‰ํ–ˆ์„ ๋•Œ eslint-config-next ๋ฒ„์ „์ด ์˜ฌ๋ผ๊ฐ€๋ฉด์„œ ๊ธฐ์กด์— ์—†๋˜ ๊ทœ์น™๋“ค์ด ์—๋Ÿฌ๋กœ ์žกํ˜€ ๋นŒ๋“œ๊ฐ€ ์‹คํŒจํ–ˆ์Šต๋‹ˆ๋‹ค.
Next.js ๊ณต์‹ ๋ฌธ์„œ(https://nextjs.org/docs/app/api-reference/config/eslint#disabling-rules) ๋ฅผ ์ฐธ๊ณ ํ•˜์—ฌ eslint.config.mjs ์— ์•„๋ž˜ ๊ทœ์น™์„ ์ถ”๊ฐ€ํ–ˆ์Šต๋‹ˆ๋‹ค.

{
  rules: {
    "react/no-unescaped-entities": "off",
    "@next/next/no-page-custom-font": "off",
  },
}

ํ…Œ์ŠคํŠธ

  • ์œ ๋‹› ํ…Œ์ŠคํŠธ ์ถ”๊ฐ€/์ˆ˜์ •๋จ
  • ๋กœ์ปฌ์—์„œ pnpm test ํ†ต๊ณผ
  • ํƒ€์ž…์ฒดํฌ/๋ฆฐํŠธ ํ†ต๊ณผ

๊ด€๋ จ ์ด์Šˆ

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

๐Ÿ”จ Refactor : mutation ์ฝ”๋“œ ๊ฐœ์„ 

1 participant